Skip to content

Conversation

@dwandro
Copy link
Contributor

@dwandro dwandro commented Dec 2, 2025

Summary

Changes

I am receiving the following error when running tests in a project that depends on the metrics package.

Error: Cannot find package '@aws/lambda-invoke-store' imported from /workspace/node_modules/.pnpm/@aws-lambda-powertools+metrics@2.29.0_@middy+core@6.4.5/node_modules/@aws-lambda-powertools/metrics/lib/esm/DimensionsStore.js

@aws/lambda-invoke-store was changed from a dependency to a devDependency of the metrics package in #4794 . Multiple packages had the version of this package updated, however the metrics package was the only one that changed it to a devDependency.

Issue number: closes #4787


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

@boring-cyborg
Copy link

boring-cyborg bot commented Dec 2, 2025

Thanks a lot for your first contribution! Please check out our contributing guidelines and don't hesitate to ask whatever you need.
In the meantime, check out the #typescript channel on our Powertools for AWS Lambda Discord: Invite link

@boring-cyborg boring-cyborg bot added the dependencies Changes that touch dependencies, e.g. Dependabot, etc. label Dec 2, 2025
@pull-request-size pull-request-size bot added the size/XS PR between 0-9 LOC label Dec 2, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 2, 2025

@svozza
Copy link
Contributor

svozza commented Dec 2, 2025

Thank you, it definitely wasn't intentional to move the package.

@svozza svozza merged commit 5e728e1 into aws-powertools:main Dec 2, 2025
52 checks passed
@boring-cyborg
Copy link

boring-cyborg bot commented Dec 2, 2025

Awesome work, congrats on your first merged pull request and thank you for helping improve everyone's experience!

@dreamorosi
Copy link
Contributor

We shouldn't have merged the PR, now the lock file is out of sync

@svozza
Copy link
Contributor

svozza commented Dec 2, 2025

i will raise a PR to update it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Changes that touch dependencies, e.g. Dependabot, etc. size/XS PR between 0-9 LOC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Maintenance: Update InvokeStore to v0.2.1

3 participants